Skip to content

Rewrite plex recommend as a native button View - #49

Merged
brah merged 1 commit into
mainfrom
modernize-recommend-view
Jun 16, 2026
Merged

Rewrite plex recommend as a native button View#49
brah merged 1 commit into
mainfrom
modernize-recommend-view

Conversation

@brah

@brah brah commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Replaces the reaction-driven selection (1️⃣2️⃣3️⃣ reactions added one-by-one with a 1s sleep between each, plus a manual wait_for("reaction_add") loop) with a native nextcord.ui.View.

  • Numbered buttons drill into each recommendation in place, with a Back to list button.
  • Defers the component interaction before fetching watchers/poster so a slow fetch can't blow the 3s ack window.
  • Clears the prior poster with attachments=[]; restricts buttons to the command author via interaction_check; disables them on timeout.
  • Drops the now-unused show_detailed_info, number_emojis, and asyncio import; updates the recommendation_timeout config description.

Verification: ruff clean, 30/30 tests pass, runtime smoke test of the view (buttons, defer, detail/overview toggle, author gate).

🤖 Generated with Claude Code

Replace the reaction-driven selection (number emojis + a wait_for
"reaction_add" loop) with a native nextcord.ui.View, matching the
pattern now used by recent/random.

The old flow added 1️⃣2️⃣3️⃣ reactions one-by-one with a 1s sleep between
each (so the menu was unusable for ~2-3s), polled wait_for in a manual
end_time loop, and posted a fresh detail message per pick (deleting the
previous). RecommendationView instead shows numbered buttons that drill
into each pick in place, with a Back button to return to the list.

Each transition defers the component interaction before fetching
watchers/poster (so a slow fetch can't blow the 3s ack window), clears
the prior poster with attachments=[], restricts the buttons to the
command author via interaction_check, and disables them on timeout.

Drops the now-unused show_detailed_info, number_emojis, and asyncio
import. Updates the recommendation_timeout config description.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@brah
brah merged commit ee0ea34 into main Jun 16, 2026
2 checks passed
@brah
brah deleted the modernize-recommend-view branch June 16, 2026 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant